Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

incus/config_template: Add file support config template create #783

Merged
merged 2 commits into from
Apr 24, 2024

Conversation

awalvie
Copy link
Contributor

@awalvie awalvie commented Apr 23, 2024

Part of #741

@awalvie awalvie requested a review from stgraber as a code owner April 23, 2024 11:18
@stgraber
Copy link
Member

This needs a rebase now :)

@awalvie awalvie force-pushed the yaml-support-config-template-create branch from 1c0c55e to 9019e12 Compare April 23, 2024 12:05
@awalvie
Copy link
Contributor Author

awalvie commented Apr 23, 2024

Re-based it! Ready for review!

@stgraber
Copy link
Member

Example should be changed as input should be a template file (pongo) and so .tpl extension, not YAML

@awalvie awalvie force-pushed the yaml-support-config-template-create branch from 2084fe5 to 5cf111e Compare April 23, 2024 13:06
@awalvie awalvie changed the title incus/config_template: Add yaml support config template create incus/config_template: Add file support config template create Apr 23, 2024
@awalvie
Copy link
Contributor Author

awalvie commented Apr 23, 2024

Example should be changed as input should be a template file (pongo) and so .tpl extension, not YAML

done!

@stgraber
Copy link
Member

Static analysis failed because the translation commit doesn't match the code.

@awalvie awalvie force-pushed the yaml-support-config-template-create branch from 98e0334 to 07a280c Compare April 24, 2024 09:59
@awalvie
Copy link
Contributor Author

awalvie commented Apr 24, 2024

Static analysis failed because the translation commit doesn't match the code.

Sorry about that, have updated them!

@awalvie awalvie force-pushed the yaml-support-config-template-create branch from 07a280c to 52b47a1 Compare April 24, 2024 10:51
@awalvie
Copy link
Contributor Author

awalvie commented Apr 24, 2024

Force pushed to re-trigger the CI.

@stgraber stgraber force-pushed the yaml-support-config-template-create branch from 52b47a1 to c7f38b3 Compare April 24, 2024 12:24
@stgraber
Copy link
Member

Github looks pretty messed up right now. I've confirmed that this is all clean and ready to merge but the Github UI won't even let me merge this yet :)

I'll try again later.

@stgraber
Copy link
Member

This needs a rebase now, hopefully Github can actually run the tests after that

@awalvie awalvie force-pushed the yaml-support-config-template-create branch from c7f38b3 to 101ec89 Compare April 24, 2024 13:45
@stgraber stgraber merged commit 033fd36 into lxc:main Apr 24, 2024
23 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants